Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handler should take clj body w/ optional headers & env as context #23

Merged

Conversation

ccfontes
Copy link
Owner

No description provided.

template/bb/index.clj Outdated Show resolved Hide resolved
template/bb/ring/middleware/json.clj Outdated Show resolved Hide resolved
template/bb/ring/middleware/json.clj Outdated Show resolved Hide resolved
template/bb/ring/middleware/json.clj Outdated Show resolved Hide resolved
template/bb/ring/middleware/json.clj Outdated Show resolved Hide resolved
template/bb/tests.clj Outdated Show resolved Hide resolved
template/bb/tests.clj Outdated Show resolved Hide resolved
template/bb/tests.clj Outdated Show resolved Hide resolved
template/bb/tests.clj Outdated Show resolved Hide resolved
template/bb/tests.clj Outdated Show resolved Hide resolved
… namespace

🔧 chore(.gitignore): ignore .clj-kondo/.cache directory
🔧 chore(template/bb/bb.edn): remove unnecessary dependencies
🔧 chore(template/bb/index.clj): refactor code to handle symbol as string
🔧 chore(template/bb/ring/middleware/json.clj): remove unused import
🔧 chore(template/bb/tests.clj): refactor tests and remove unused code
🔧 chore(template/bb/tests.clj): add fixture for response in tests
🔧 chore(template/bb/tests.clj): refactor tests to use fixture and add TODO comment
🔧 chore(template/bb/tests.clj): add main function to run tests
template/bb/tests.clj Outdated Show resolved Hide resolved
🐛 fix(index.clj): fix indentation and remove unused imports in index.clj file
🐛 fix(index.clj): fix arity-2-handler function to destructure correctly
🐛 fix(index.clj): fix ->handler function to correctly merge response
🐛 fix(tests.clj): fix example in index/->kebab-case test
🐛 fix(tests.clj): fix arity-2-handler function to destructure correctly
🐛 fix(tests.clj): fix app test to use resp-fixture
🚧 chore(index.clj): add TODO comments for future tasks
📝 docs(README.adoc): update project description and add third party code section
✨ feat(examples): add bb-map-context example
🔧 chore(stack.yml): add bb-http-map-context function to stack
🔧 chore(template): add author, url, and license metadata to index.clj and tests.clj
🔧 chore(template): add author, contributors, url, and license metadata to json.clj
…to 1.0.15

🔧 chore(faas_fn_build_invoke.yml): update bb-http-function build and invoke steps
🔧 chore(faas_fn_build_invoke.yml): update bb-streaming-hello and bb-streaming-lib e2e tests
🔧 chore(faas_fn_build_invoke.yml): update bb-http-map and bb-http-map-context e2e tests
🔧 chore(template/bb/index.clj): remove TODO comment
🔧 chore(template/bb/ring/middleware/json.clj): fix metadata indentation
🔧 chore(template/bb/tests.clj): add handler arity tests
…ge version

🔧 chore(faas_fn_build_invoke.yml): add docker logs for debugging purposes
📝 docs(README.adoc): fix typo in the documentation
🐛 fix(handler.clj): fix handler function to accept and return content instead of a map
… stop/remove containers

🔧 chore(faas_fn_build_invoke.yml): stop and remove bb-http-map and bb-http-map-context containers
…running new ones

🔧 chore(faas_fn_build_invoke.yml): stop and remove bb-http-map container before running new one
🔧 chore(faas_fn_build_invoke.yml): stop and remove bb-http-map-context container before running new one
…ntainers

The sleep command is added to allow some time for the containers to start before making requests to them. This helps to ensure that the containers are fully initialized and ready to handle requests.
…e container

✨ feat(faas_fn_build_invoke.yml): improve error handling for container startup
…rameter

🐛 fix(handler.clj): print context variable for debugging purposes
@ccfontes ccfontes merged commit fd06989 into main Aug 20, 2023
@ccfontes ccfontes deleted the handler-should-take-clj-body-w-optional-headers-env-context branch August 20, 2023 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant